Skip to content

Studio: Improve context for errors that bubble up from wordpress-server-child#3477

Open
katinthehatsite wants to merge 9 commits into
trunkfrom
add/make-child-process-errors-bubble-up
Open

Studio: Improve context for errors that bubble up from wordpress-server-child#3477
katinthehatsite wants to merge 9 commits into
trunkfrom
add/make-child-process-errors-bubble-up

Conversation

@katinthehatsite
Copy link
Copy Markdown
Contributor

@katinthehatsite katinthehatsite commented May 13, 2026

Related issues

Fixes STU-1496

How AI was used in this PR

It was used to help identify the issue.

Proposed Changes

This PR ensures that we display more detailed error message to the user from wordpress-server-child

Testing Instructions

  • Pull the changes from this branch
  • Start Studio with npm start
  • Use the broken blueprint e.g.
    broken-blueprint.json
  • Click on Add site -> Build a new site -> Upload blueprint
  • Observe that you can see more details about why the upload failed:
Screenshot 2026-05-15 at 11 12 11 AM
  • Confirm that when you open Studio logs, you can also find more details there:
Screenshot 2026-05-15 at 11 12 23 AM

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@katinthehatsite katinthehatsite self-assigned this May 13, 2026
@katinthehatsite katinthehatsite marked this pull request as draft May 13, 2026 18:45
@wpmobilebot
Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 6183398 vs trunk

app-size

Metric trunk 6183398 Diff Change
App Size (Mac) 1409.60 MB 1409.46 MB 0.14 MB ⚪ 0.0%

site-editor

Metric trunk 6183398 Diff Change
load 1526 ms 1489 ms 37 ms ⚪ 0.0%

site-startup

Metric trunk 6183398 Diff Change
siteCreation 8573 ms 8581 ms +8 ms ⚪ 0.0%
siteStartup 4927 ms 4931 ms +4 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@katinthehatsite katinthehatsite requested a review from a team May 15, 2026 09:41
@katinthehatsite katinthehatsite marked this pull request as ready for review May 15, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants